Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

normalize proxy properties #1650

Merged
merged 2 commits into from Oct 10, 2016
Merged

normalize proxy properties #1650

merged 2 commits into from Oct 10, 2016

Conversation

msimerson
Copy link
Member

@msimerson msimerson commented Oct 9, 2016

Do the same for proxy properties as #1098 did for the rest of the connection properties.

Changes proposed in this pull request:

  • c.haproxy_ip -> c.proxy.ip
  • c.proxy -> c.proxy.allowed
  • c.proxy_time -> c.proxy.timer
  • new property: c.proxy.type (null, haproxy, someFutureProxy)

Checklist:

  • docs updated
  • tests updated

@msimerson msimerson mentioned this pull request Oct 10, 2016
2 tasks
@codecov-io
Copy link

Current coverage is 34.53% (diff: 25.00%)

Merging #1650 into master will decrease coverage by 0.01%

@@             master      #1650   diff @@
==========================================
  Files            25         25          
  Lines          6385       6388     +3   
  Methods         825        825          
  Messages          0          0          
  Branches       1665       1668     +3   
==========================================
  Hits           2206       2206          
- Misses         4179       4182     +3   
  Partials          0          0          

Powered by Codecov. Last update 57288f7...f4a91bc

@msimerson msimerson merged commit b3ce582 into haraka:master Oct 10, 2016
@msimerson msimerson deleted the normalize-proxy-settings branch October 10, 2016 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants